Skip to content

BUG: pandas.cut and negative values #14652 #14663

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 6 commits into from

Conversation

luca-s
Copy link
Contributor

@luca-s luca-s commented Nov 15, 2016

@jreback
Copy link
Contributor

jreback commented Nov 15, 2016

tests pls

@jreback
Copy link
Contributor

jreback commented Nov 15, 2016

pls add an entry in whatsnew for 0.19.2

@jreback jreback added Bug Groupby Reshaping Concat, Merge/Join, Stack/Unstack, Explode labels Nov 15, 2016
@codecov-io
Copy link

codecov-io commented Nov 16, 2016

Current coverage is 85.28% (diff: 100%)

Merging #14663 into master will not change coverage

@@             master     #14663   diff @@
==========================================
  Files           140        140          
  Lines         50693      50693          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits          43234      43234          
  Misses         7459       7459          
  Partials          0          0          

Powered by Codecov. Last update 3f523f3...8db26db

@luca-s
Copy link
Contributor Author

luca-s commented Nov 16, 2016

Please let me know if that's ok because I wasn't sure where to add the test case.

@@ -25,3 +25,4 @@ Bug Fixes

- compat with ``dateutil==2.6.0`` for testing (:issue:`14621`)
- allow ``nanoseconds`` in ``Timestamp.replace`` kwargs (:issue:`14621`)
- fix ``pandas.cut`` with one negative value and one bin (:issue:`14652`)
Copy link
Contributor

@jreback jreback Nov 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug in pd.cut


expected = pd.Series([0, 0])

s = pd.Series([9., 9.])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pandas\tools\tests\test_tile.py

lots of tests there, just add this one on., no need to create a new class. add a comment with the issue number

@luca-s
Copy link
Contributor Author

luca-s commented Nov 17, 2016

Thanks @jreback, now it should be fine.

@jreback jreback added this to the 0.19.2 milestone Nov 17, 2016
@jreback
Copy link
Contributor

jreback commented Nov 17, 2016

thanks!

@jreback jreback closed this in 2fc0c68 Nov 17, 2016
@luca-s luca-s deleted the issue_14652 branch November 17, 2016 13:33
@luca-s luca-s restored the issue_14652 branch November 17, 2016 13:34
jorisvandenbossche pushed a commit to jorisvandenbossche/pandas that referenced this pull request Dec 14, 2016
…s-dev#14652

closes pandas-dev#14652

Author: Luca Scarabello <luca.scarabello@sed.ethz.ch>
Author: Luca <luca.scarabello@gmail.com>

Closes pandas-dev#14663 from luca-s/issue_14652 and squashes the following commits:

8db26db [Luca Scarabello] Moved new test location to pandas\tools\tests\test_tile.py
90dd07d [Luca Scarabello] Updated whatsnew
d6b3da8 [Luca Scarabello] fixed flake8 compliance
fdc55b9 [Luca Scarabello] Added test case for pandas-dev#14652
d5790e2 [Luca Scarabello] updated whatsnew v0.19.2
2db0c7a [Luca] BUG: pandas.cut and negative values pandas-dev#14652

(cherry picked from commit 2fc0c68)
@luca-s luca-s deleted the issue_14652 branch February 17, 2017 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Groupby Reshaping Concat, Merge/Join, Stack/Unstack, Explode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: pandas.cut and negative values
3 participants